Release 0.1.7 by siy · Pull Request #8 · siy/java-peglib

siy · 2025-12-30T12:32:15Z

Summary

JBCT compliance refactoring: replaced null with Option throughout codebase
Added jbct-maven-plugin 0.4.1 for code formatting and linting
Added 34 new internal type tests (271 → 305 tests)

Changes

ParseResultWithDiagnostics.node() returns Option<CstNode>
ParseMode uses Option for nullable fields
ParsingContext packrat cache uses Option
Diagnostic.code() returns Option<String>
Generated ParseResult and CstParseResult use Option for nullable fields

Test plan

All 305 tests pass
JBCT check passes (0 issues)
Build succeeds

🤖 Generated with Claude Code

Summary by CodeRabbit

Chores
- Released version 0.1.7 (2025-12-30) with synchronized version references in changelog, project documentation, and build configuration files.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

coderabbitai · 2025-12-30T12:32:23Z

📝 Walkthrough

Walkthrough

Version bump from 0.1.7 to 0.1.7 across project configuration and documentation files, including the Maven POM, changelog, and README dependency reference. No functional changes or logic modifications.

Changes

Cohort / File(s)	Summary
Version Release Updates `CHANGELOG.md`, `README.md`, `pom.xml`	Updated version identifiers from 0.1.6 to 0.1.7 across Maven POM coordinates, changelog header, and documentation dependency version.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A new release hops into the light,
Version bumps from six to seven bright!
Pom and docs now synchronized true,
0.1.7—our latest breakthrough! ✨

Pre-merge checks

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title 'Release 0.1.7' is a generic version label that does not capture the actual substantial changes made in this PR, which involve JBCT compliance refactoring (replacing null with Option) and adding test coverage.	Consider a more descriptive title like 'Refactor to use Option for null safety and improve test coverage' to better communicate the primary technical changes to reviewers.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 32f9008 and 50565e7.

📒 Files selected for processing (3)

CHANGELOG.md
README.md
pom.xml

🧰 Additional context used

📓 Path-based instructions (2)

**/*.md

📄 CodeRabbit inference engine (CLAUDE.md)

Documentation should include grammar syntax reference, API usage examples, implementation progress checklist, and comprehensive test coverage summary

Files:

CHANGELOG.md
README.md

pom.xml

📄 CodeRabbit inference engine (CLAUDE.md)

Maven configuration must target Java 25 and depend on pragmatica-lite:core 0.9.0 for Result/Option/Promise types

Files:

pom.xml

🧠 Learnings (6)

📓 Common learnings

Learnt from: CR
Repo: siy/java-peglib PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-12-27T13:57:49.345Z
Learning: Applies to src/main/java/org/pragmatica/peg/generator/*.java : Source generator must produce self-contained single-file Java parsers with optional ErrorReporting mode (BASIC/ADVANCED)